gtk: Fix memleaks
authorBenjamin Otte <otte@redhat.com>
Tue, 13 Dec 2016 23:06:01 +0000 (00:06 +0100)
committerBenjamin Otte <otte@redhat.com>
Tue, 20 Dec 2016 17:01:10 +0000 (18:01 +0100)
It turns out, some simple getters - such as
gdk_drawing_context_get_clip() - love copying things before returning
them.

I guess somebody has to burn cycles...

gtk/gtkwidget.c
gtk/inspector/recorder.c

index 85783162bb901364f41d255b1a3b8ced8abff2d1..163f621a8ba5fade2acd1044f2b1a146d125d802 100644 (file)
@@ -15616,6 +15616,7 @@ gtk_widget_render (GtkWidget            *widget,
   GtkSnapshot snapshot;
   GskRenderer *renderer;
   GskRenderNode *root;
+  cairo_region_t *clip;
 
   /* We only render double buffered on native windows */
   if (!gdk_window_has_native (window))
@@ -15626,11 +15627,13 @@ gtk_widget_render (GtkWidget            *widget,
     return;
 
   context = gsk_renderer_begin_draw_frame (renderer, region);
+  clip = gdk_drawing_context_get_clip (context);
 
   gtk_snapshot_init (&snapshot,
                      renderer,
-                     gdk_drawing_context_get_clip (context),
+                     clip,
                      "Render<%s>", G_OBJECT_TYPE_NAME (widget));
+  cairo_region_destroy (clip);
   gtk_widget_snapshot (widget, &snapshot);
   root = gtk_snapshot_finish (&snapshot);
   if (root != NULL)
@@ -15646,5 +15649,6 @@ gtk_widget_render (GtkWidget            *widget,
       gsk_render_node_unref (root);
     }
 
+
   gsk_renderer_end_draw_frame (renderer, context);
 }
index bffab8393abe6757cd254727db7ed50b6d4baba5..1a0f5d1fce3e4955ceb7ca87d67b16e8969fbaab 100644 (file)
@@ -461,11 +461,13 @@ gtk_inspector_recorder_record_render (GtkInspectorRecorder *recorder,
 {
   GtkInspectorRecording *recording;
   GdkFrameClock *frame_clock;
+  cairo_region_t *clip;
 
   if (!gtk_inspector_recorder_is_recording (recorder))
     return;
 
   frame_clock = gtk_widget_get_frame_clock (widget);
+  clip = gdk_drawing_context_get_clip (context);
 
   recording = gtk_inspector_render_recording_new (gdk_frame_clock_get_frame_time (frame_clock),
                                                   gsk_renderer_get_profiler (renderer),
@@ -473,10 +475,11 @@ gtk_inspector_recorder_record_render (GtkInspectorRecorder *recorder,
                                                     gdk_window_get_width (window),
                                                     gdk_window_get_height (window) },
                                                   region,
-                                                  gdk_drawing_context_get_clip (context),
+                                                  clip,
                                                   node);
   gtk_inspector_recorder_add_recording (recorder, recording);
   g_object_unref (recording);
+  cairo_region_destroy (clip);
 }
 
 // vim: set et sw=2 ts=2: